feat: governance scripts #1019
feat: governance scripts #1019
Conversation
gov/scripts/.gitignore
Outdated
| @@ -0,0 +1,3 @@ | |||
| /node_modules/ | |||
| .yarn/cache | |||
| .yarn/install-state.gz | |||
There was a problem hiding this comment.
.yarn/install-state.gz не нужно исключать, если у тебя стоит старый yarn
А новый yarn можно установить в проект только локально, смотри .yarnrc.yml в tests.
Но вместо того, чтобы ставить его сюда... Предлагаю переместить эти скрипты в тесты.
Это позволит избавиться ещё от lock файла лишнего.
Как насчёт JS часть прибрать, и разбить на пакеты?
Завести js-packages, а там
js-packages/types (Бывший types/src/interfaces)
js-packages/tests (Остальное содержимое тестов)
js-packages/playgrounds (Плейграунды, чтобы их было проще в чистоте поддерживать)
js-packages/scripts (Вынести туда калибрацию и всё прочее для мейнтененса, в т.ч governance)
Я работал с yarn3 воркспейсами, и могу сделать основную часть этого рефакторинга
There was a problem hiding this comment.
Подобное разнесение уже давно напрашивается для JS части
There was a problem hiding this comment.
Также хочется некоторые утилиты тесно связанные с нашими тестами и чейном там разместить, чтобы веб команда в том числе могла их переиспользовать без копипасты, но при этом мы могли там размещать сложную логику, которую имеет смысл поддерживать одновременно с чейном. Потенциально кодирование будущего RBAC надо бы где-то разместить, поскольку битфилды в polkadot.js поддерживаются никак.
There was a problem hiding this comment.
В тесты это класть как-то не хочется. Я мог вообще отдельный репо под это дело завести, на самом деле.
Оно только для удобства governance-related повторяемых действий
There was a problem hiding this comment.
Не хочется иметь несколько больших локфайлов и использовать несколько разных версий yarn, у нас и без того гитхаб жалуется на уязвимости)
Если нет необходимости хранить в этой репе - то отдельная тоже хорошо. А скрипты в тестах... Они уже есть, и поэтому сейчас у нас директория тестов уже является js-барахолкой
| const councilProposalThreshold = Math.floor(councilMembers.length / 2) + 1; | ||
|
|
||
| const democracyProposal = api.tx.democracy.externalProposeDefault({ | ||
| Inline: authorizeUpgrade.method.toHex(), |
There was a problem hiding this comment.
Не совсем понимаю, а в чём отличие между ISubmittable.toHex() от ISubmittable.method.toHex()?
Действий не требует, просто обычно я напрямую делал
No description provided.